Merged
Conversation
Contributor
There was a problem hiding this comment.
Greptile Overview
Summary
This PR updates the landing page video and social proof sections with new content and improved UX. The main changes replace the demo video with a new launch video, consolidate dual light/dark preview images into a single image, enhance social testimonials with clickable links and custom avatars, and improve responsive design by hiding the Book a Demo button on smaller screens.
Key changes:
- Replaced
DEMO_VIDEO_URLwith newLAUNCH_VIDEO_URLconstant in video component - Consolidated dual theme video preview images into single
preview_video.webp - Enhanced social testimonials with optional
urlandavatarfields, enabling clickable cards - Added custom avatar for Alex (OpenSea Co-founder) testimonial
- Updated user role labels from "Reddit User"/"Discord User" to "Hacker News"/"Discord"
- Increased icon size (to 5) and text size (to lg) in social stats component
- Added responsive
max-xl:hiddenclass to navbar Book a Demo button
Confidence Score: 5/5
- This PR is safe to merge with minimal risk
- All changes are straightforward content and styling updates with no logical errors, security issues, or breaking changes. The code follows existing patterns, properly imports the new constant, and maintains backward compatibility.
- No files require special attention
Important Files Changed
File Analysis
| Filename | Score | Overview |
|---|---|---|
| pcweb/constants.py | 5/5 | Added new LAUNCH_VIDEO_URL constant for updated video content |
| pcweb/pages/landing/views/video.py | 5/5 | Updated video component to use new LAUNCH_VIDEO_URL and single preview image |
| pcweb/pages/landing/views/social_marquee.py | 5/5 | Enhanced social testimonial cards with optional URLs, custom avatars, and updated user roles |
| pcweb/pages/landing/views/social_stats.py | 5/5 | Updated icon size and text styling in stats display |
| pcweb/components/docpage/navbar/navbar.py | 5/5 | Added max-xl:hidden class to hide Book a Demo button on smaller screens |
Sequence Diagram
sequenceDiagram
participant User
participant Landing Page
participant Video Component
participant Social Marquee
participant Constants
participant YouTube
User->>Landing Page: Visit landing page
Landing Page->>Video Component: Render video_demo()
Video Component->>Constants: Import LAUNCH_VIDEO_URL
Constants-->>Video Component: Return video URL
Video Component->>Video Component: Display preview_video.webp
User->>Video Component: Click play button
Video Component->>YouTube: Load LAUNCH_VIDEO_URL
YouTube-->>Video Component: Stream video content
Video Component-->>User: Display video in dialog
Landing Page->>Social Marquee: Render social_marquee()
Social Marquee->>Social Marquee: Create social cards with URLs
Social Marquee-->>Landing Page: Display testimonials
User->>Social Marquee: Click testimonial card
Social Marquee->>External Site: Navigate to testimonial URL
External Site-->>User: Open Hacker News/Discord link
5 files reviewed, no comments
Alek99
approved these changes
Oct 21, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.